Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

making a clone of _decl_props. #207

Merged
merged 1 commit into from Jul 14, 2012
Merged

making a clone of _decl_props. #207

merged 1 commit into from Jul 14, 2012

Conversation

cfitz
Copy link
Contributor

@cfitz cfitz commented Jul 14, 2012

Apologize about the second request..but this makes a clone of the _decl_props that is updated.

@andreasronge
Copy link
Member

Thanks, btw I you have time it would be great to use this column thing in a to_s class method for the Neo4j::Rails::Model.
That would be very useful when working in the IRB console which is also how active record work.

andreasronge added a commit that referenced this pull request Jul 14, 2012
making a clone of _decl_props.
@andreasronge andreasronge merged commit d759f27 into neo4jrb:master Jul 14, 2012
@cfitz
Copy link
Contributor Author

cfitz commented Jul 19, 2012

Yeah, looking at this, it definitely could be better...i'm going to start refactoring the activerecord column class to work with the neo orm...

@andreasronge
Copy link
Member

Great ! Maybe you can also implement the inspect method on the Neo4j::Rails::Model and Neo4j::Rails::Relationship so that it works like the active record model inspect method. That would be useful when running the rails console.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants